The expectedModels array is missing the updated Gemini preview models (gemini-3-flash-preview, gemini-3-pro-preview, gemini-3.1-pro-preview) which were added to the debate skill's quick reference table. Including them ensures that the test verifies the full set of current models.

Section 15 of the tests verifies the consult skill adapter sync but completely omits the Gemini model updates mentioned in the PR description. Adding assertions for the new Gemini preview models will ensure that both the OpenCode adapter and the canonical consult skill are correctly updated.

describe('consult skill opencode adapter sync (#232)', () => {
  test('opencode consult adapter contains all 5 providers', () => {
    for (const provider of ['claude', 'gemini', 'codex', 'opencode', 'copilot']) {
      expect(openCodeConsultSkillContent.toLowerCase()).toContain(provider);
    }
  });

  test('opencode consult adapter has updated claude model names', () => {
    expect(openCodeConsultSkillContent).toContain('claude-haiku-4-5');
    expect(openCodeConsultSkillContent).toContain('claude-sonnet-4-6');
    expect(openCodeConsultSkillContent).toContain('claude-opus-4-6');
  });

  test('opencode consult adapter has updated gemini model names', () => {
    expect(openCodeConsultSkillContent).toContain('gemini-3-flash-preview');
    expect(openCodeConsultSkillContent).toContain('gemini-3-pro-preview');
    expect(openCodeConsultSkillContent).toContain('gemini-3.1-pro-preview');
  });

  test('opencode consult adapter has updated codex model names (no speculative gpt-5.x)', () => {
    expect(openCodeConsultSkillContent).not.toContain('gpt-5.3-codex');
    expect(openCodeConsultSkillContent).not.toContain('gpt-5.2-codex');
    expect(openCodeConsultSkillContent).toContain('o4-mini');
    expect(openCodeConsultSkillContent).toContain('o3');
  });

  test('canonical consult skill has updated model names', () => {
    expect(consultSkillContent).toContain('claude-haiku-4-5');
    expect(consultSkillContent).toContain('claude-sonnet-4-6');
    expect(consultSkillContent).toContain('claude-opus-4-6');
    expect(consultSkillContent).toContain('gemini-3-flash-preview');
    expect(consultSkillContent).toContain('gemini-3-pro-preview');
    expect(consultSkillContent).toContain('gemini-3.1-pro-preview');
    expect(consultSkillContent).not.toContain('gpt-5.3-codex');
    expect(consultSkillContent).toContain('o4-mini');
    expect(consultSkillContent).toContain('o3');
  });
});

The Codex safe command pattern uses nested unescaped double quotes inside a double-quoted command substitution ("$(cat "{AI_STATE_DIR}/...")"), which is not valid shell syntax. Update the pattern to avoid nested unescaped quotes (e.g., use single quotes around the temp-file path or escape inner quotes) and keep it consistent with the canonical consult skill table.

The Codex safe command pattern contains nested unescaped double quotes inside a double-quoted command substitution, which makes the example invalid if copied. Adjust the quoting (e.g., single-quote the temp-file path or escape inner quotes) and ensure it matches the canonical consult skill safe-pattern table.

The Codex safe command pattern example has invalid shell quoting due to nested unescaped double quotes around the temp-file path. Rewrite the example to avoid nested quotes (single quotes or escaped quotes) so the reference is syntactically correct.

The Codex safe command pattern is not valid shell as written because it includes nested unescaped double quotes inside a double-quoted command substitution. Please fix the quoting so the example can be copied safely.

The Codex safe command pattern uses nested unescaped double quotes ("$(cat "{AI_STATE_DIR}/...")"), which breaks shell parsing. Update the quoting (e.g., single-quote the file path or escape inner quotes) to make the reference command syntactically valid.

Grammar: use “an External Tool Quick Reference section” (External starts with a vowel sound).

The new “consult skill adapter sync” tests only validate the OpenCode adapter + canonical consult skill. The Codex adapter consult skill still contains stale model names (e.g., gpt-5.3-codex in adapters/codex/skills/consult/SKILL.md), so the repo can remain inconsistent while these tests pass. Consider updating the Codex adapter consult docs (and/or extending this test to cover it) to ensure the model-name update is applied consistently across platforms.

Update saved-session model example to renamed Claude IDs

This change switches Claude defaults to fully qualified model IDs (claude-*), but the same skill still documents writing "model": "opus" in last-session.json and then reusing that saved model on --continue (see the state-file schema and load-session steps). That leaves contradictory instructions in one prompt and can make resumed consultations use stale model names instead of the newly documented defaults, especially when the agent follows the schema literally.

Useful? React with 👍 / 👎.